-
Notifications
You must be signed in to change notification settings - Fork 266
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix redis test #15247
Fix redis test #15247
Conversation
e9f5c96
to
4f64f8a
Compare
5915da9
to
dd70013
Compare
b9e035a
to
29b5a6d
Compare
data/console/redis_cli.sh
Outdated
# Connect to the new instance of redis-server with port 6380 and make 6380 instance | ||
#a replica of redis instance running on port 6379 and verify the reslt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor comments:
and make 6380 -> delete the blank
verify the reslt -> result?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correction is done. Thanks for the heads up
tests/console/redis.pm
Outdated
sub run_script { | ||
my $script = shift; | ||
my $logfile = "output.txt"; | ||
record_info($script, "Running python script: $script"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO, it should be a shell script, but please omit my comment if I am wrong
data/console/redis_cli.sh
Outdated
echo "redis-cli info replication on master" | ||
else | ||
echo "Failed: redis-cli info replication on master" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete this line
data/console/redis_cli.sh
Outdated
echo "Role:slave in redis-cli on replica" | ||
else | ||
echo "Failed: redis-cli info replication on replica" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete this line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deleted. Thanks Richard
e2336fc
to
22220d2
Compare
data/console/redis_cli.sh
Outdated
then | ||
echo "redis-cli info replication on master" | ||
else | ||
echo "Failed: redis-cli info replication on master" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
echo "Failed: redis-cli info replication on master" | |
echo "Failed: redis-cli info replication on master, slave took longer than 10 seconds to show up" |
out=$(redis-cli -p 6380 HMGET "movie:343" title) | ||
if [ "$out" = "Spider-Man" ] | ||
then | ||
echo "redis-cli replicaof done" | ||
else | ||
echo "Failed: redis-cli replication on replica " | ||
exit 1 | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you create a follow up ticket to rework this in the future?, i.e: add an extra check to know the status of the replication (basically, know if the replica is in sync with the master)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
created ticket: https://progress.opensuse.org/issues/115592
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from the niptick of one of the messages, everything looks good
SLE 15 SP3 - x86_64 | s390x| | aarch64
SLE 15 SP2 - x86_64 | s390x| | aarch64
SLE 15 SP4 - x86_64 | ppc64le| | aarch64 | svirt-xen-hvm | svirt-xen-pv | s390x
LEAP: x86_64
Tumbleweed: x86_64